Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve caching of artifacts using new Transport#downloadArtifact #3771

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Apr 21, 2024

Currently Tycho maintain different caches, e.g. some items are cached in the local maven repository, some are cached in the transport caches and there is even a P2 remote cache maintained. Apart from that, P2/PDE/Ooomph also maintain so called bundle pools.

With the new Transport#downloadArtifact method we now have an opportunity to merge all these caches in a reliable way just using the artifact that is already there saving both downloads times and disk space.

Currently Tycho maintain different caches, e.g. some items are cached in
the local maven repository, some are cached in the transport caches and
there is even a P2 remote cache maintained. Apart from that,
P2/PDE/Ooomph also maintain so called bundle pools.

With the new Transport#downloadArtifact method we now have an
opportunity to merge all these caches in a reliable way just using the
artifact that is already there saving both downloads times and disk
space.
Copy link

Test Results

  594 files    594 suites   3h 48m 37s ⏱️
  418 tests   411 ✅  7 💤 0 ❌
1 254 runs  1 232 ✅ 22 💤 0 ❌

Results for commit ef64935.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant